Skip to content

Add Shadeform integration#29

Merged
drewmalin merged 15 commits intobrevdev:mainfrom
shadeform:main
Aug 28, 2025
Merged

Add Shadeform integration#29
drewmalin merged 15 commits intobrevdev:mainfrom
shadeform:main

Conversation

@cholical
Copy link
Collaborator

This is a PR that adds the Shadeform integration for Brev

@drewmalin
Copy link
Contributor

@cholical looks good!

One thing I noticed (that @theFong also seems to have run into here: 360a505) is the use of the openapi generation made an incorrect import statement. Could you make the following change?

diff --git a/internal/shadeform/gen/shadeform/test/api_default_test.go b/internal/shadeform/gen/shadeform/test/api_default_test.go
index 2ba4390..ed79488 100644
--- a/internal/shadeform/gen/shadeform/test/api_default_test.go
+++ b/internal/shadeform/gen/shadeform/test/api_default_test.go
@@ -11,19 +11,18 @@ package openapi

 import (
        "context"
-       openapiclient "github.com/brevdev/cloud"
+       "testing"
+
+       openapiclient "github.com/brevdev/cloud/internal/shadeform/gen/shadeform"
        "github.com/stretchr/testify/assert"
        "github.com/stretchr/testify/require"
-       "testing"
 )

@drewmalin drewmalin merged commit 4048bdd into brevdev:main Aug 28, 2025
1 of 4 checks passed
@drewmalin
Copy link
Contributor

Will follow up with the change noted above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants